Skip to content

Conversation

lo92fr
Copy link
Contributor

@lo92fr lo92fr commented Oct 20, 2025

Fix sanetization of channel type UID

This PR is related to issue #19505.
Channel type UID are not properly sanetize.
If the sort descriptor contains specials character like "°", we end up with an exception.

…annelTypeUid are correct

Signed-off-by: Laurent ARNAL <laurent@clae.net>
@lsiepel lsiepel added the bug An unexpected problem or unintended behavior of an add-on label Oct 20, 2025
@lsiepel lsiepel changed the title [Siemenshvac] Fix sanetization of channel type UID [siemenshvac] Fix sanetization of channel type UID Oct 20, 2025
Signed-off-by: Laurent ARNAL <laurent@clae.net>
Signed-off-by: Laurent ARNAL <laurent@clae.net>
Signed-off-by: Laurent ARNAL <laurent@clae.net>
Copy link
Contributor

@lsiepel lsiepel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM

@lsiepel lsiepel merged commit 5198ed7 into openhab:main Oct 20, 2025
2 checks passed
@lsiepel lsiepel added this to the 5.1 milestone Oct 20, 2025
lsiepel pushed a commit that referenced this pull request Oct 20, 2025
* adding call to sanetizeid into normalizedescriptor to be sure that channelTypeUid are correct

Signed-off-by: Laurent ARNAL <laurent@clae.net>
@lsiepel lsiepel added the backported A PR that has been cherry-picked to a patch release branch label Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backported A PR that has been cherry-picked to a patch release branch bug An unexpected problem or unintended behavior of an add-on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[SiemensHVAC] missing sanitization in the binding’s UID generation while bridge is initializating

2 participants